Add a failing test for addrparse_header #78
Conversation
Mailbox name is only decoded if it is first in the list. In the test, the second mailbox name is not decoded.
| @@ -1062,5 +1062,26 @@ mod tests { | |||
| .unwrap() | |||
| )]) | |||
| ); | |||
|
|
|||
| let (parsed, _) = crate::parse_header( | |||
| b"To: foo <foo@example.org>,?UTF-8?B?Zm9v8J+Qm2Jhcg==?= <bar@example.org>", | |||
There was a problem hiding this comment.
Is it intentional that there's no = between the , and the ?UTF bits? Because that's a pretty bad spec violation and I wouldn't want to try and detect that. Even if I add the =, mailparse doesn't handle this correctly, so there's still a bug that I need to fix, but I wanted to confirm that your original thing has the = character and the omission here is a typo.
|
I pushed a fix for the case where there is an I'm going to close this PR since I took this patch and pushed it along with my fix; I made some tweaks to it but left you as the author on it. |
|
No, missing |
Mailbox name is only decoded if it is first in the list. In the test,
the second mailbox name is not decoded.